Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flag valid measure status in iOS backing views to avoid repeat measuring #12627

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Jan 13, 2023

Description of Change

The iOS backing views for xplat layout are blindly re-measuring during each LayoutSubviews call. This change adds a flag when measure has occurred so that it doesn't get repeated. The flag is cleared on each invalidation.

Issues Fixed

Drastically reduces the number of measure calls during layout on iOS.

@rmarinho rmarinho merged commit 514ebb0 into main Jan 18, 2023
@rmarinho rmarinho deleted the reduce-ios-measure branch January 18, 2023 11:25
@symbiogenesis
Copy link
Contributor

Will this make it into the .NET 7.0 branch?

@rjhind
Copy link

rjhind commented Jan 24, 2023

I hope it does. It was mentioned as part of #11140 along with #12660 (which was back-ported)

@PureWeen
Copy link
Member

/backport to net7.0

@github-actions
Copy link
Contributor

Started backporting to net7.0: https://github.com/dotnet/maui/actions/runs/4016751513

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! platform/iOS 🍎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants